Conversation
|
@bolom Then how do you invoke Brakeman? Another solution would be to stick to Brakeman 5.0. We don't need the latest version when we use old Ruby and Rails versions. As long as it supports Rails 5.0 this is fine. We'll upgrade it along with other dependencies as we continue to upgrade. |
We could maybe add a new action in Github action |
|
Yeah, let's get Rubocop merged, and we can run Brakeman right after it. |
|
@ysbaddaden not sure what I done wrong. every thing is red now :( |
|
Looking at the CI logs, it complains about Nokogiri::HTML4 being undefined. Looking at Nokogiri's documentation:
There is an invalid dependency. I see Loofah got upgraded and Nokogiri's stuck at 1.10. We don't want to upgrade Nokogiri without continuing with the Ruby/Rails upgrade (CDx uses Nokogiri directly for a XML user feature). |
Brakeman doesn't support Ruby 2.4, as suggested by the gem's author in this conversation, I've used Brakeman-lib